Skip to content

Revert "BLD: pin numpy to particular variant that is built for all our deps #17625

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 22, 2017

Conversation

jreback
Copy link
Contributor

@jreback jreback commented Sep 22, 2017

xref #17619

This reverts commit 6930f27.

closes #17620

@jreback jreback added the CI Continuous Integration label Sep 22, 2017
@jreback jreback added this to the 0.21.0 milestone Sep 22, 2017
@jreback jreback changed the title Revert "BLD: pin numpy to particular variant that is built for all ou… Revert "BLD: pin numpy to particular variant that is built for all our deps Sep 22, 2017
@jreback
Copy link
Contributor Author

jreback commented Sep 22, 2017

looks like this now builds correctly (with the reverted pin). though it does downgrade numpy/blas appropriately, so I guess conda-forge fixed the dep issue.

@jreback
Copy link
Contributor Author

jreback commented Sep 22, 2017

cc @jorisvandenbossche @gfyoung

@jreback jreback merged commit d1fe892 into pandas-dev:master Sep 22, 2017
@jorisvandenbossche
Copy link
Member

it does downgrade numpy/blas appropriately,

Yes, but it only does downgrade the blas version, not numpy version, so that doesn't affect the ABI compatibility of pandas. So indeed the deps issue seems fixed by conda-forge (there should have been some package that lacked numpy 1.13 build forcing it to downgrade)

Thanks for the follow-up!

alanbato pushed a commit to alanbato/pandas that referenced this pull request Nov 10, 2017
No-Stream pushed a commit to No-Stream/pandas that referenced this pull request Nov 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI: reverse pin on numpy
2 participants